SPM: Move xlat-related functions to separate file
authorAntonio Nino Diaz <[email protected]>
Tue, 22 May 2018 15:45:35 +0000 (16:45 +0100)
committerAntonio Nino Diaz <[email protected]>
Wed, 23 May 2018 12:50:18 +0000 (13:50 +0100)
This is done in order to make it easier to read the file spm_main.c.

Change-Id: I21e765154c1682a319a3bc47a19a42fd736e910e
Signed-off-by: Antonio Nino Diaz <[email protected]>
services/std_svc/spm/sp_xlat.c [new file with mode: 0644]
services/std_svc/spm/spm.mk
services/std_svc/spm/spm_main.c
services/std_svc/spm/spm_private.h

diff --git a/services/std_svc/spm/sp_xlat.c b/services/std_svc/spm/sp_xlat.c
new file mode 100644 (file)
index 0000000..b192b0c
--- /dev/null
@@ -0,0 +1,155 @@
+/*
+ * Copyright (c) 2018, ARM Limited and Contributors. All rights reserved.
+ *
+ * SPDX-License-Identifier: BSD-3-Clause
+ */
+
+#include <arch.h>
+#include <arch_helpers.h>
+#include <assert.h>
+#include <errno.h>
+#include <platform_def.h>
+#include <platform.h>
+#include <secure_partition.h>
+#include <spm_svc.h>
+#include <xlat_tables_v2.h>
+
+#include "spm_private.h"
+#include "spm_shim_private.h"
+
+/* Place translation tables by default along with the ones used by BL31. */
+#ifndef PLAT_SP_IMAGE_XLAT_SECTION_NAME
+#define PLAT_SP_IMAGE_XLAT_SECTION_NAME        "xlat_table"
+#endif
+
+/* Allocate and initialise the translation context for the secure partitions. */
+REGISTER_XLAT_CONTEXT2(sp,
+                      PLAT_SP_IMAGE_MMAP_REGIONS,
+                      PLAT_SP_IMAGE_MAX_XLAT_TABLES,
+                      PLAT_VIRT_ADDR_SPACE_SIZE, PLAT_PHY_ADDR_SPACE_SIZE,
+                      EL1_EL0_REGIME, PLAT_SP_IMAGE_XLAT_SECTION_NAME);
+
+/* Lock used for SP_MEMORY_ATTRIBUTES_GET and SP_MEMORY_ATTRIBUTES_SET */
+static spinlock_t mem_attr_smc_lock;
+
+/* Get handle of Secure Partition translation context */
+xlat_ctx_t *spm_get_sp_xlat_context(void)
+{
+       return &sp_xlat_ctx;
+};
+
+/*
+ * Attributes are encoded using a different format in the SMC interface than in
+ * the Trusted Firmware, where the mmap_attr_t enum type is used. This function
+ * converts an attributes value from the SMC format to the mmap_attr_t format by
+ * setting MT_RW/MT_RO, MT_USER/MT_PRIVILEGED and MT_EXECUTE/MT_EXECUTE_NEVER.
+ * The other fields are left as 0 because they are ignored by the function
+ * change_mem_attributes().
+ */
+static unsigned int smc_attr_to_mmap_attr(unsigned int attributes)
+{
+       unsigned int tf_attr = 0U;
+
+       unsigned int access = (attributes & SP_MEMORY_ATTRIBUTES_ACCESS_MASK)
+                             >> SP_MEMORY_ATTRIBUTES_ACCESS_SHIFT;
+
+       if (access == SP_MEMORY_ATTRIBUTES_ACCESS_RW) {
+               tf_attr |= MT_RW | MT_USER;
+       } else if (access ==  SP_MEMORY_ATTRIBUTES_ACCESS_RO) {
+               tf_attr |= MT_RO | MT_USER;
+       } else {
+               /* Other values are reserved. */
+               assert(access ==  SP_MEMORY_ATTRIBUTES_ACCESS_NOACCESS);
+               /* The only requirement is that there's no access from EL0 */
+               tf_attr |= MT_RO | MT_PRIVILEGED;
+       }
+
+       if ((attributes & SP_MEMORY_ATTRIBUTES_NON_EXEC) == 0) {
+               tf_attr |= MT_EXECUTE;
+       } else {
+               tf_attr |= MT_EXECUTE_NEVER;
+       }
+
+       return tf_attr;
+}
+
+/*
+ * This function converts attributes from the Trusted Firmware format into the
+ * SMC interface format.
+ */
+static unsigned int smc_mmap_to_smc_attr(unsigned int attr)
+{
+       unsigned int smc_attr = 0U;
+
+       unsigned int data_access;
+
+       if ((attr & MT_USER) == 0) {
+               /* No access from EL0. */
+               data_access = SP_MEMORY_ATTRIBUTES_ACCESS_NOACCESS;
+       } else {
+               if ((attr & MT_RW) != 0) {
+                       assert(MT_TYPE(attr) != MT_DEVICE);
+                       data_access = SP_MEMORY_ATTRIBUTES_ACCESS_RW;
+               } else {
+                       data_access = SP_MEMORY_ATTRIBUTES_ACCESS_RO;
+               }
+       }
+
+       smc_attr |= (data_access & SP_MEMORY_ATTRIBUTES_ACCESS_MASK)
+                   << SP_MEMORY_ATTRIBUTES_ACCESS_SHIFT;
+
+       if ((attr & MT_EXECUTE_NEVER) != 0U) {
+               smc_attr |= SP_MEMORY_ATTRIBUTES_NON_EXEC;
+       }
+
+       return smc_attr;
+}
+
+int32_t spm_memory_attributes_get_smc_handler(secure_partition_context_t *sp_ctx,
+                                             uintptr_t base_va)
+{
+       uint32_t attributes;
+
+       spin_lock(&mem_attr_smc_lock);
+
+       int rc = get_mem_attributes(sp_ctx->xlat_ctx_handle,
+                                    base_va, &attributes);
+
+       spin_unlock(&mem_attr_smc_lock);
+
+       /* Convert error codes of get_mem_attributes() into SPM ones. */
+       assert((rc == 0) || (rc == -EINVAL));
+
+       if (rc == 0) {
+               return (int32_t) smc_mmap_to_smc_attr(attributes);
+       } else {
+               return SPM_INVALID_PARAMETER;
+       }
+}
+
+int spm_memory_attributes_set_smc_handler(secure_partition_context_t *sp_ctx,
+                                         u_register_t page_address,
+                                         u_register_t pages_count,
+                                         u_register_t smc_attributes)
+{
+       uintptr_t base_va = (uintptr_t) page_address;
+       size_t size = (size_t) (pages_count * PAGE_SIZE);
+       uint32_t attributes = (uint32_t) smc_attributes;
+
+       INFO("  Start address  : 0x%lx\n", base_va);
+       INFO("  Number of pages: %i (%zi bytes)\n", (int) pages_count, size);
+       INFO("  Attributes     : 0x%x\n", attributes);
+
+       spin_lock(&mem_attr_smc_lock);
+
+       int ret = change_mem_attributes(sp_ctx->xlat_ctx_handle,
+                                       base_va, size,
+                                       smc_attr_to_mmap_attr(attributes));
+
+       spin_unlock(&mem_attr_smc_lock);
+
+       /* Convert error codes of change_mem_attributes() into SPM ones. */
+       assert((ret == 0) || (ret == -EINVAL));
+
+       return (ret == 0) ? SPM_SUCCESS : SPM_INVALID_PARAMETER;
+}
index 39431054f70fe5b0fb51eb8ad2deea40e43dd946..0e77086045c7ea5f9eb166628659c8cf9915a9e7 100644 (file)
@@ -15,7 +15,8 @@ SPM_SOURCES   :=      $(addprefix services/std_svc/spm/,      \
                        ${ARCH}/spm_helpers.S                   \
                        ${ARCH}/spm_shim_exceptions.S           \
                        spm_main.c                              \
-                       sp_setup.c)
+                       sp_setup.c                              \
+                       sp_xlat.c)
 
 
 # Let the top-level Makefile know that we intend to include a BL32 image
index 0c88c63cfec4dc990173bbc69435e23a706d4a6e..133eea295cca2e840fbbf6ca83a516ed3a4f80f3 100644 (file)
 #include <xlat_tables_v2.h>
 
 #include "spm_private.h"
-#include "spm_shim_private.h"
-
-/* Place translation tables by default along with the ones used by BL31. */
-#ifndef PLAT_SP_IMAGE_XLAT_SECTION_NAME
-#define PLAT_SP_IMAGE_XLAT_SECTION_NAME        "xlat_table"
-#endif
-
-/* Allocate and initialise the translation context for the secure partitions. */
-REGISTER_XLAT_CONTEXT2(sp,
-                      PLAT_SP_IMAGE_MMAP_REGIONS,
-                      PLAT_SP_IMAGE_MAX_XLAT_TABLES,
-                      PLAT_VIRT_ADDR_SPACE_SIZE, PLAT_PHY_ADDR_SPACE_SIZE,
-                      EL1_EL0_REGIME, PLAT_SP_IMAGE_XLAT_SECTION_NAME);
-
-/* Lock used for SP_MEMORY_ATTRIBUTES_GET and SP_MEMORY_ATTRIBUTES_SET */
-static spinlock_t mem_attr_smc_lock;
-
-/* Get handle of Secure Partition translation context */
-xlat_ctx_t *spm_get_sp_xlat_context(void)
-{
-       return &sp_xlat_ctx;
-};
 
 /*******************************************************************************
  * Secure Partition context information.
@@ -131,124 +109,6 @@ int32_t spm_setup(void)
        return 0;
 }
 
-/*
- * Attributes are encoded using a different format in the SMC interface than in
- * the Trusted Firmware, where the mmap_attr_t enum type is used. This function
- * converts an attributes value from the SMC format to the mmap_attr_t format by
- * setting MT_RW/MT_RO, MT_USER/MT_PRIVILEGED and MT_EXECUTE/MT_EXECUTE_NEVER.
- * The other fields are left as 0 because they are ignored by the function
- * change_mem_attributes().
- */
-static unsigned int smc_attr_to_mmap_attr(unsigned int attributes)
-{
-       unsigned int tf_attr = 0U;
-
-       unsigned int access = (attributes & SP_MEMORY_ATTRIBUTES_ACCESS_MASK)
-                             >> SP_MEMORY_ATTRIBUTES_ACCESS_SHIFT;
-
-       if (access == SP_MEMORY_ATTRIBUTES_ACCESS_RW) {
-               tf_attr |= MT_RW | MT_USER;
-       } else if (access ==  SP_MEMORY_ATTRIBUTES_ACCESS_RO) {
-               tf_attr |= MT_RO | MT_USER;
-       } else {
-               /* Other values are reserved. */
-               assert(access ==  SP_MEMORY_ATTRIBUTES_ACCESS_NOACCESS);
-               /* The only requirement is that there's no access from EL0 */
-               tf_attr |= MT_RO | MT_PRIVILEGED;
-       }
-
-       if ((attributes & SP_MEMORY_ATTRIBUTES_NON_EXEC) == 0) {
-               tf_attr |= MT_EXECUTE;
-       } else {
-               tf_attr |= MT_EXECUTE_NEVER;
-       }
-
-       return tf_attr;
-}
-
-/*
- * This function converts attributes from the Trusted Firmware format into the
- * SMC interface format.
- */
-static unsigned int smc_mmap_to_smc_attr(unsigned int attr)
-{
-       unsigned int smc_attr = 0U;
-
-       unsigned int data_access;
-
-       if ((attr & MT_USER) == 0) {
-               /* No access from EL0. */
-               data_access = SP_MEMORY_ATTRIBUTES_ACCESS_NOACCESS;
-       } else {
-               if ((attr & MT_RW) != 0) {
-                       assert(MT_TYPE(attr) != MT_DEVICE);
-                       data_access = SP_MEMORY_ATTRIBUTES_ACCESS_RW;
-               } else {
-                       data_access = SP_MEMORY_ATTRIBUTES_ACCESS_RO;
-               }
-       }
-
-       smc_attr |= (data_access & SP_MEMORY_ATTRIBUTES_ACCESS_MASK)
-                   << SP_MEMORY_ATTRIBUTES_ACCESS_SHIFT;
-
-       if ((attr & MT_EXECUTE_NEVER) != 0U) {
-               smc_attr |= SP_MEMORY_ATTRIBUTES_NON_EXEC;
-       }
-
-       return smc_attr;
-}
-
-static int32_t spm_memory_attributes_get_smc_handler(
-                                       secure_partition_context_t *sp_ctx,
-                                       uintptr_t base_va)
-{
-       uint32_t attributes;
-
-       spin_lock(&mem_attr_smc_lock);
-
-       int rc = get_mem_attributes(sp_ctx->xlat_ctx_handle,
-                                    base_va, &attributes);
-
-       spin_unlock(&mem_attr_smc_lock);
-
-       /* Convert error codes of get_mem_attributes() into SPM ones. */
-       assert((rc == 0) || (rc == -EINVAL));
-
-       if (rc == 0) {
-               return (int32_t) smc_mmap_to_smc_attr(attributes);
-       } else {
-               return SPM_INVALID_PARAMETER;
-       }
-}
-
-static int spm_memory_attributes_set_smc_handler(
-                                       secure_partition_context_t *sp_ctx,
-                                       u_register_t page_address,
-                                       u_register_t pages_count,
-                                       u_register_t smc_attributes)
-{
-       uintptr_t base_va = (uintptr_t) page_address;
-       size_t size = (size_t) (pages_count * PAGE_SIZE);
-       uint32_t attributes = (uint32_t) smc_attributes;
-
-       INFO("  Start address  : 0x%lx\n", base_va);
-       INFO("  Number of pages: %i (%zi bytes)\n", (int) pages_count, size);
-       INFO("  Attributes     : 0x%x\n", attributes);
-
-       spin_lock(&mem_attr_smc_lock);
-
-       int ret = change_mem_attributes(sp_ctx->xlat_ctx_handle,
-                                       base_va, size,
-                                       smc_attr_to_mmap_attr(attributes));
-
-       spin_unlock(&mem_attr_smc_lock);
-
-       /* Convert error codes of change_mem_attributes() into SPM ones. */
-       assert((ret == 0) || (ret == -EINVAL));
-
-       return (ret == 0) ? SPM_SUCCESS : SPM_INVALID_PARAMETER;
-}
-
 /*******************************************************************************
  * Secure Partition Manager SMC handler.
  ******************************************************************************/
index 0d2c2ee8983fdad96376327fb048b155b42a14fb..28bf6f17113741ffd936f89db4cb891e2c239be9 100644 (file)
@@ -49,6 +49,15 @@ void __dead2 spm_secure_partition_exit(uint64_t c_rt_ctx, uint64_t ret);
 
 void secure_partition_setup(secure_partition_context_t *sp_ctx);
 
+xlat_ctx_t *spm_get_sp_xlat_context(void);
+
+int32_t spm_memory_attributes_get_smc_handler(secure_partition_context_t *sp_ctx,
+                                             uintptr_t base_va);
+int spm_memory_attributes_set_smc_handler(secure_partition_context_t *sp_ctx,
+                                         u_register_t page_address,
+                                         u_register_t pages_count,
+                                         u_register_t smc_attributes);
+
 #endif /* __ASSEMBLY__ */
 
 #endif /* __SPM_PRIVATE_H__ */